-
Notifications
You must be signed in to change notification settings - Fork 120
emsdk 5.0.0 #506
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
emsdk 5.0.0 #506
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. WalkthroughExpanded workflow pull_request globs to recursive patterns; bumped wasm package version and removed its prebuild patch; switched native backend init to use AnySurfaceState; added Emscripten linker rustflags for wasm; removed two unused serde default helpers; updated emsdk version and submodule pointer. Changes
Sequence Diagram(s)(omitted — changes do not introduce a new multi-component sequential flow requiring visualization) Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…cess to prevent bundling of Node built-ins
-sENVIRONMENT=web,node) breaks Next.js Turbopack client builds due to Node built-insrequire("fs")/require("path")in generated glue emscripten-core/emscripten#26134 (fixed)Summary by CodeRabbit
Chores
Refactor
Note
✏️ Tip: You can customize this high-level summary in your review settings.